Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Noticeboard Parser #1

Open
wants to merge 74 commits into
base: main
Choose a base branch
from
Open

Noticeboard Parser #1

wants to merge 74 commits into from

Conversation

Bikram-ghuku
Copy link

@Bikram-ghuku Bikram-ghuku commented Oct 4, 2024

Description

Initial part of the project. Uses erp ssoToken to login and access the internal noticeboard in erp. Looks for any new messages every 2 mins and prints them.

Features

  • Notification publisher, checker every two minutes
  • Implement NTFY push service
  • REALLY good logging
  • Dockerise application
  • Create a monitoring service
  • Use IIT ERP login package

@Bikram-ghuku Bikram-ghuku marked this pull request as ready for review October 6, 2024 06:45
@proffapt proffapt self-requested a review October 6, 2024 07:32
@Bikram-ghuku
Copy link
Author

Bikram-ghuku commented Oct 6, 2024

Suggested changes

  • REALLY good logging
  • Dockerise application
  • Create a monitoring service
  • Use IIT ERP login package

.env.example Outdated Show resolved Hide resolved
mfins-doctor/go.mod Outdated Show resolved Hide resolved
mfins/erplogin/login.go Outdated Show resolved Hide resolved
@harshkhandeparkar
Copy link
Member

@shikharish please review.

Copy link
Member

@proffapt proffapt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the ERP login package. If not working, figure it out and make it work.

mfins/.dockerignore Outdated Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ayein?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was Initialising the monitoring service :)

mfins/metaploy/PROJECT_NAME.metaploy.conf Outdated Show resolved Hide resolved
@proffapt
Copy link
Member

Good work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

3 participants